Skip to content

copyedit gui->testing #19

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 5, 2025
Merged

copyedit gui->testing #19

merged 1 commit into from
Jun 5, 2025

Conversation

Tom-Willemsen
Copy link
Member

@Tom-Willemsen Tom-Willemsen commented May 20, 2025

Copyedit GUI -> Testing section. Summary of changes below.


Adding unit tests

  • General sentence-level/formatting changes

An introduction to unit testing

  • General sentence-level/formatting changes
  • Run through all examples to check they still work, update where they didn't
  • Update instructions and screenshots for newer versions of Eclipse

Mockito

  • Run through examples to check they work, update where they didn't
  • Corrections/updates for newer Mockito APIs
    • e.g. recommend @Mock over Mockito.mock() as it's type-safe with generics
  • Link much more prominently to authoritative upstream documentation
    • I'm not actually sure we really want to keep much of this page - it's mostly a duplicate of the (higher-quality) upstream documentation. But have kept for now as it's "only" a few hundred lines of duplication.

Squish

  • General formatting improvements
  • Correct various dead links

Squish BDD tools

  • General formatting improvements

Test naming

  • General sentence-level/formatting changes
  • No substantive change to the documented conventions
  • Correct misleading example

@Tom-Willemsen Tom-Willemsen force-pushed the gui_testing_copyedit branch from 980053b to efd7f23 Compare May 20, 2025 17:10
@Tom-Willemsen Tom-Willemsen marked this pull request as ready for review May 29, 2025 08:50
@Tom-Willemsen Tom-Willemsen moved this to Review in PI_2025_02 May 29, 2025
@rerpha rerpha merged commit e48a9ce into master Jun 5, 2025
2 checks passed
@github-project-automation github-project-automation bot moved this from Review to Done in PI_2025_02 Jun 5, 2025
@rerpha rerpha deleted the gui_testing_copyedit branch June 5, 2025 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants